Skip to content

Conversation

IDisposable
Copy link
Contributor

Simply ignores any IPv6 addresses in the lease and doesn't offer them to the RPC
Fixes #685

@IDisposable IDisposable force-pushed the feature/disable-ipv6 branch 2 times, most recently from a67f641 to fb2ae84 Compare August 27, 2025 16:13
Simply ignores any IPv6 addresses in the lease and doesn't offer them to the RPC
Also fixed display issue for IPv6 link local address.
Fixes https://github.com/orgs/jetkvm/projects/7/views/1?pane=issue&itemId=122761546&issue=jetkvm%7Ckvm%7C685
@IDisposable
Copy link
Contributor Author

Rebased and resolved conflicts

@adamshiervani
Copy link
Contributor

Doesn't work for me; I can still access the UI over IPv6 after disabling it in the settings.

Let's just wait until #732 gets merged, and then add this. A lot will change in #732 in regard to networking, and it will just be easier to base it off that work.

@IDisposable
Copy link
Contributor Author

Yep, would be best. I wasn't really sure of the scope of this change and saw the other one in flight.

@IDisposable
Copy link
Contributor Author

@adamshiervani I cleaned this code up a lot, perhaps it's good enough to ship now?

@IDisposable
Copy link
Contributor Author

(I can't reopen, and the PR is not updating :( ) started a new one at #803

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants